Fix RPATH wrapper by detecting & replacing -Xlinker --enable-new-dtags#4970
Fix RPATH wrapper by detecting & replacing -Xlinker --enable-new-dtags#4970boegel merged 4 commits intoeasybuilders:developfrom
-Xlinker --enable-new-dtags#4970Conversation
…-enable-new-dtags` The linker flag `-Xlinker --enable-new-dtags` was not detected by the `rpath_args.py` script, causing the use of RUNPATH instead of RPATH in some cases. In particular, this happened with Intel MPI compiler wrappers (mpiicpc), that always add `--Xlinker --enable-new-dtags` to the compilation command.
lorisercole
force-pushed
the
fix/rpath-intel
branch
from
d0a0c30 to
5dfd3a1
Compare
boegel
changed the title
-Xlinker --enable-new-dtags
Co-authored-by: Kenneth Hoste <kenneth.hoste@ugent.be>
|
Just thinking, perhaps we can extend this PR to also address this issue: As @Micket commented easybuilders/easybuild-easyconfigs#23031 (comment) , is there a reason to add RPATHs flags when the compiler is invoked with |
We should address this, as this causes issues with at least Intel + Meson (like I also just commented in that issue). However, I think we should handle this in a separate, focused, PR. |
Fully agreed, smaller focused PRs are easier to review/merge |
3 participants
Fix #4969 RPATH wrapper: detect and replace
-Xlinker --enable-new-dtagsThe linker flag
-Xlinker --enable-new-dtagswas not detected by therpath_args.pyscript, causing the use of RUNPATH instead of RPATH in some cases.In particular, this happened with Intel MPI compiler wrappers (e.g.
mpiicpc), that always add--Xlinker --enable-new-dtagsto the compilation command.I also added some minor improvements to the script.